Skip to content

fix(OP-TEE): configs: Fix OPTEE_TARGET_FLAVOUR#296

Merged
praneethbajjuri merged 1 commit into
TexasInstruments:masterfrom
manorit2001:optee-fix-platform
May 28, 2025
Merged

fix(OP-TEE): configs: Fix OPTEE_TARGET_FLAVOUR#296
praneethbajjuri merged 1 commit into
TexasInstruments:masterfrom
manorit2001:optee-fix-platform

Conversation

@manorit2001

Copy link
Copy Markdown
Contributor

This is wrong for some platforms, update them to fix the builds.

This is wrong for some platforms, update them to fix the builds.

Reported-by: Diwakar Dhyani <d-dhyani@ti.com>
Signed-off-by: Manorit Chawdhry <m-chawdhry@ti.com>
@manorit2001 manorit2001 force-pushed the optee-fix-platform branch from ca15f91 to 0071fd4 Compare May 19, 2025 10:37
@praneethbajjuri

Copy link
Copy Markdown
Collaborator

Adding @bryanbrattlof as well for review.
Is this change necessary after pulling optee update that contains this OP-TEE/optee_os#7377 ?

@bryanbrattlof

bryanbrattlof commented May 19, 2025

Copy link
Copy Markdown
Collaborator

ah yeah I didn't notice the Jacinto chips had the same issue 🫤 but this looks great 👍

62L: https://github.com/TexasInstruments/processor-sdk-doc/blob/master/configs/AM62LX/AM62LX_linux_config.txt#L27
62P: https://github.com/TexasInstruments/processor-sdk-doc/blob/master/configs/AM62PX/AM62PX_linux_config.txt#L27

are still a little messed up but we can fix that later

@praneethbajjuri

Copy link
Copy Markdown
Collaborator

are still a little messed up but we can fix that later

Or wait till OPTEE sha/tag is updated that contains the fix and no need to update doc after that.

@manorit2001

Copy link
Copy Markdown
Contributor Author

are still a little messed up but we can fix that later

Or wait till OPTEE sha/tag is updated that contains the fix and no need to update doc after that.

I would still recommend to not make it the norm to add such new cosmetic platforms btw, since it's already in it's fine but please fix the documentation only next time onwards as otherwise it can become a mess.

@praneethbajjuri praneethbajjuri merged commit 3e8ecbf into TexasInstruments:master May 28, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants